Skip to content

Comments

[souliss] Fix channel metadata#19597

Merged
lolodomo merged 2 commits intoopenhab:mainfrom
lsiepel:souliss-channels
Nov 3, 2025
Merged

[souliss] Fix channel metadata#19597
lolodomo merged 2 commits intoopenhab:mainfrom
lsiepel:souliss-channels

Conversation

@lsiepel
Copy link
Contributor

@lsiepel lsiepel commented Oct 31, 2025

While improving all bindings documentation and thing / channel structures. Some issues showed with the souliss binding.

  1. Channel type buttonReadOnly had non-matching labels and description
  2. Thing t58 had the wrong channel type

Can be backported to 5.0.x

Signed-off-by: Leo Siepel <leosiepel@gmail.com>
@lsiepel lsiepel requested a review from fazioa as a code owner October 31, 2025 15:09
@lsiepel lsiepel added the bug An unexpected problem or unintended behavior of an add-on label Oct 31, 2025
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question about upgrade instructions

Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you

@lolodomo lolodomo merged commit d61c00f into openhab:main Nov 3, 2025
2 checks passed
@lolodomo lolodomo added this to the 5.1 milestone Nov 3, 2025
@lsiepel lsiepel deleted the souliss-channels branch November 3, 2025 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An unexpected problem or unintended behavior of an add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants